Skip to content

ci: added release please config #112

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Apr 1, 2025
Merged

ci: added release please config #112

merged 16 commits into from
Apr 1, 2025

Conversation

twishabansal
Copy link
Contributor

@twishabansal twishabansal commented Mar 26, 2025

Generated release PRs look like: #113

Additionally, functionality has been implemented to ensure that distinct release PRs are created for separate packages.

@twishabansal twishabansal changed the title ci: add release please config ci: test release please config Mar 26, 2025
@twishabansal twishabansal changed the title ci: test release please config ci: added release please config Mar 26, 2025
@twishabansal
Copy link
Contributor Author

Generated release PRs look like: #113

@twishabansal twishabansal marked this pull request as ready for review March 26, 2025 11:28
@twishabansal twishabansal requested a review from a team as a code owner March 26, 2025 11:28
@twishabansal twishabansal requested a review from kurtisvg March 26, 2025 11:28
@twishabansal twishabansal requested a review from kurtisvg March 27, 2025 05:07
@kurtisvg kurtisvg force-pushed the release-config-test branch from d2acf5b to 2d29611 Compare March 31, 2025 15:35
@twishabansal twishabansal merged commit 2b2d736 into main Apr 1, 2025
13 checks passed
@twishabansal twishabansal deleted the release-config-test branch April 1, 2025 05:25
twishabansal added a commit that referenced this pull request Apr 1, 2025
* ci: add release please config

* chore: add initial version

* chore: specify initial version as string

* chore: Update .release-please-manifest.json

* chore: add empty json

* chore: small change

* chore: try fixing config

* chore: try fixing config again

* chore: remove release-as

* chore: add changelog sections

* chore: better release notes

* chore: better release notes

* chore: change toolbox-langchain version

* chore: separate PRs for packages

* chore: change PR style
twishabansal added a commit that referenced this pull request Apr 3, 2025
* feat: add authenticated parameters support

* chore: add asyncio dep

* chore: run itest

* chore: add type hint

* fix: call tool instead of client

* chore: correct arg name

* feat: add support for bound parameters

* chore: add tests for bound parameters

* docs: update syntax error on readme (#121)

* ci: added release please config (#112)

* ci: add release please config

* chore: add initial version

* chore: specify initial version as string

* chore: Update .release-please-manifest.json

* chore: add empty json

* chore: small change

* chore: try fixing config

* chore: try fixing config again

* chore: remove release-as

* chore: add changelog sections

* chore: better release notes

* chore: better release notes

* chore: change toolbox-langchain version

* chore: separate PRs for packages

* chore: change PR style

* added basic e2e tests

* change license year

* add test deps

* fix tests

* fix tests

* fix tests

* add new test case

* fix docstring

* added todo

* cleanup

* add bind param test case

* make bind params dynamic

* try fix test errors

* lint

* remove redundant test

* test fix

* fix docstring

* feat: add authenticated parameters support

* chore: add asyncio dep

* chore: run itest

* chore: add type hint

* fix: call tool instead of client

* chore: correct arg name

* chore: address feedback

* chore: address more feedback

* feat: add support for bound parameters

* chore: add tests for bound parameters

* chore: address feedback

* revert package file changes

* fix error message

* revert package files

* lint

* fix error message

* Update packages/toolbox-core/tests/test_e2e.py

Co-authored-by: Anubhav Dhawan <[email protected]>

* add new test case

* change docstring to reflect new test cases

* clean up docstring

* lint

* Move tests to different classes

* add timeout

---------

Co-authored-by: Kurtis Van Gent <[email protected]>
Co-authored-by: Yuan <[email protected]>
Co-authored-by: Anubhav Dhawan <[email protected]>
twishabansal added a commit that referenced this pull request Apr 3, 2025
… and their descriptions (#127)

* feat: add authenticated parameters support

* chore: add asyncio dep

* chore: run itest

* chore: add type hint

* fix: call tool instead of client

* chore: correct arg name

* feat: add support for bound parameters

* chore: add tests for bound parameters

* docs: update syntax error on readme (#121)

* ci: added release please config (#112)

* ci: add release please config

* chore: add initial version

* chore: specify initial version as string

* chore: Update .release-please-manifest.json

* chore: add empty json

* chore: small change

* chore: try fixing config

* chore: try fixing config again

* chore: remove release-as

* chore: add changelog sections

* chore: better release notes

* chore: better release notes

* chore: change toolbox-langchain version

* chore: separate PRs for packages

* chore: change PR style

* added basic e2e tests

* change license year

* add test deps

* fix tests

* fix tests

* fix tests

* add new test case

* fix docstring

* added todo

* cleanup

* add bind param test case

* make bind params dynamic

* try fix test errors

* lint

* remove redundant test

* test fix

* fix docstring

* feat: add authenticated parameters support

* chore: add asyncio dep

* chore: run itest

* chore: add type hint

* fix: call tool instead of client

* chore: correct arg name

* chore: address feedback

* chore: address more feedback

* feat: add support for bound parameters

* chore: add tests for bound parameters

* chore: address feedback

* revert package file changes

* fix error message

* revert package files

* lint

* fix error message

* Update packages/toolbox-core/tests/test_e2e.py

Co-authored-by: Anubhav Dhawan <[email protected]>

* try changing docstring to include args and their descriptions

* fix docstring

* lint

* fix test

* lint

* added return type annotation

* Add docstrings

* use and update schema

# Conflicts:
#	packages/toolbox-core/src/toolbox_core/tool.py

* lint

* lint

* revert changes to e2e test file

# Conflicts:
#	packages/toolbox-core/tests/test_e2e.py

* change string to str and in arg type

* small change

* fix imports

* remove toolschema usage

* lint

* move create_docstring method outside the class.

* lint

---------

Co-authored-by: Kurtis Van Gent <[email protected]>
Co-authored-by: Yuan <[email protected]>
Co-authored-by: Anubhav Dhawan <[email protected]>
twishabansal added a commit that referenced this pull request Apr 3, 2025
* feat: add authenticated parameters support

* chore: add asyncio dep

* chore: run itest

* chore: add type hint

* fix: call tool instead of client

* chore: correct arg name

* feat: add support for bound parameters

* chore: add tests for bound parameters

* docs: update syntax error on readme (#121)

* ci: added release please config (#112)

* ci: add release please config

* chore: add initial version

* chore: specify initial version as string

* chore: Update .release-please-manifest.json

* chore: add empty json

* chore: small change

* chore: try fixing config

* chore: try fixing config again

* chore: remove release-as

* chore: add changelog sections

* chore: better release notes

* chore: better release notes

* chore: change toolbox-langchain version

* chore: separate PRs for packages

* chore: change PR style

* added basic e2e tests

* change license year

* add test deps

* fix tests

* fix tests

* fix tests

* add new test case

* fix docstring

* added todo

* cleanup

* add bind param test case

* make bind params dynamic

* try fix test errors

* lint

* remove redundant test

* test fix

* fix docstring

* feat: add authenticated parameters support

* chore: add asyncio dep

* chore: run itest

* chore: add type hint

* fix: call tool instead of client

* chore: correct arg name

* chore: address feedback

* chore: address more feedback

* feat: add support for bound parameters

* chore: add tests for bound parameters

* chore: address feedback

* revert package file changes

* fix error message

* revert package files

* lint

* fix error message

* Update packages/toolbox-core/tests/test_e2e.py

Co-authored-by: Anubhav Dhawan <[email protected]>

* try changing docstring to include args and their descriptions

* fix docstring

* lint

* fix test

* lint

* added return type annotation

* Add docstrings

* use and update schema

# Conflicts:
#	packages/toolbox-core/src/toolbox_core/tool.py

* lint

* lint

* try out pydantic validation

# Conflicts:
#	packages/toolbox-core/src/toolbox_core/client.py
#	packages/toolbox-core/src/toolbox_core/tool.py

* revert changes to e2e test file

* add basic pydantic type checking

* lint

* fix pydantic validation error

* fix error string

* fix error string

* lint

* added to_pydantic_model as a method under the toolschema class

* lint

* revert changes to e2e test file

# Conflicts:
#	packages/toolbox-core/tests/test_e2e.py

* change string to str and in arg type

* small change

* fix imports

* remove toolschema usage

* lint

* create pydantic model at init

* lint

* move create_docstring method outside the class.

* lint

* lint

* Update packages/toolbox-core/src/toolbox_core/tool.py

Co-authored-by: Anubhav Dhawan <[email protected]>

* move to_pydantic_model outside the class

* lint

* rename function

* added name to pydantic model

* lint

---------

Co-authored-by: Kurtis Van Gent <[email protected]>
Co-authored-by: Yuan <[email protected]>
Co-authored-by: Anubhav Dhawan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants